Skip to content

refactor(skills): rename skills with category prefixes (setup-/security-)#31

Merged
potiuk merged 1 commit into
apache:mainfrom
potiuk:refactor/rename-skills-with-category-prefixes
May 2, 2026
Merged

refactor(skills): rename skills with category prefixes (setup-/security-)#31
potiuk merged 1 commit into
apache:mainfrom
potiuk:refactor/rename-skills-with-category-prefixes

Conversation

@potiuk

@potiuk potiuk commented May 2, 2026

Copy link
Copy Markdown
Member

Summary

  • Rename 13 skills with setup- / security- category prefixes (flat-on-disk — Claude Code only discovers flat .claude/skills/<name>/, nested subdirs verified empirically as not picked up).
  • Drop apache- from *-apache-steward skill names (pre-empts the framework rename to apache/steward); pluralise batch-style skills (security-import-issues, security-sync-issues, security-deduplicate-issues).
  • Update cross-references in 46 files (README, AGENTS.md, CONTRIBUTING.md, secure-agent-setup.md, secure-agent-internals.md, projects/_template/, tools/, generate-cve-json docstrings). 353/353 insertions/deletions — pure rename, no semantic change.

Preparation for adding the pr-* skill family (pr-triage, pr-stats, pr-maintainer-review) lifted from apache/airflow in a follow-up PR.

Rename table

Old New
setup-secure-config (unchanged — already prefixed)
verify-secure-config setup-verify-secure-config
update-secure-config setup-update-secure-config
upgrade-apache-steward setup-upgrade-steward
verify-apache-steward setup-verify-steward
sync-shared-config setup-sync-shared-config
import-security-issue security-import-issues
import-security-issue-from-md security-import-issues-from-md
import-security-issue-from-pr security-import-issue-from-pr
sync-security-issue security-sync-issues
fix-security-issue security-fix-issue
allocate-cve security-allocate-cve
deduplicate-security-issue security-deduplicate-issues
invalidate-security-issue security-invalidate-issue

Breaking change for adopters

Adopters using the framework via submodule + symlinks (e.g. the airflow-s tracker) need to:

  1. Bump their submodule pointer to the merge commit of this PR.
  2. Update any project-side .claude/skills/<name> symlinks to the new names.

Test plan

  • Claude Code discovers all renamed skills (live-change detection picked them up in-session)
  • No bare references to old names remain in tracked files (grep audit)
  • No double-prefix collisions (setup-setup-, security-security-)
  • Pre-commit hook passes locally
  • CI link-check (lychee) passes
  • Reviewer eyeballs the rename table for plural/singular consistency

🤖 Generated with Claude Code

…ty-)

Group skills by function via name prefix rather than directory nesting
(Claude Code only discovers flat .claude/skills/<name>/SKILL.md).

Setup-lifecycle skills get a `setup-` prefix:

  verify-secure-config       → setup-verify-secure-config
  update-secure-config       → setup-update-secure-config
  upgrade-apache-steward     → setup-upgrade-steward
  verify-apache-steward      → setup-verify-steward
  sync-shared-config         → setup-sync-shared-config

(setup-secure-config keeps its name — already prefixed.)

Security workflow skills get a `security-` prefix, drop the now-
redundant `security` suffix, and pluralise the noun for batch-style
skills:

  import-security-issue          → security-import-issues
  import-security-issue-from-md  → security-import-issues-from-md
  import-security-issue-from-pr  → security-import-issue-from-pr
  sync-security-issue            → security-sync-issues
  fix-security-issue             → security-fix-issue
  allocate-cve                   → security-allocate-cve
  deduplicate-security-issue     → security-deduplicate-issues
  invalidate-security-issue      → security-invalidate-issue

Cross-references updated everywhere: README, AGENTS.md,
CONTRIBUTING.md, secure-agent-setup.md, secure-agent-internals.md,
how-to-fix-a-security-issue.md, new-members-onboarding.md,
projects/_template/, tools/, plus generate-cve-json docstrings.

Preparation for adding pr-* skill family (pr-triage, pr-stats,
pr-maintainer-review) lifted from apache/airflow in a follow-up PR.

Generated-by: Claude Code (Claude Opus 4.7)
@potiuk potiuk merged commit 3c69d96 into apache:main May 2, 2026
7 checks passed
@andreahlert andreahlert added the mode:platform Substrate / infra — not a mode (sandbox, CI, validators) label May 7, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

mode:platform Substrate / infra — not a mode (sandbox, CI, validators)

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants